<html>
<head><meta charset="utf-8"><title>Uplift `temporary-cstring-as-ptr` lint fr… compiler-team#346 · t-compiler/major changes · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/index.html">t-compiler/major changes</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html">Uplift `temporary-cstring-as-ptr` lint fr… compiler-team#346</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="207295093"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/207295093" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> triagebot <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#207295093">(Aug 18 2020 at 18:17)</a>:</h4>
<p>A new proposal has been announced: <a href="https://github.com/rust-lang/compiler-team/issues/346">#346</a>. It will be announced at the next meeting to try and draw attention to it, but usually MCPs are not discussed during triage meetings. If you think this would benefit from discussion amongst the team, consider proposing a design meeting.</p>



<a name="207296062"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/207296062" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Jonas Schievink  [he/him] <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#207296062">(Aug 18 2020 at 18:24)</a>:</h4>
<p>note that it has at least one known false positive: <a href="https://github.com/rust-lang/rust-clippy/issues/5838">https://github.com/rust-lang/rust-clippy/issues/5838</a></p>



<a name="207299293"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/207299293" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#207299293">(Aug 18 2020 at 18:50)</a>:</h4>
<p>I do wonder how hard it would be to treat the raw pointer returned from as_ptr as a reference for borrow checking, though perhaps just lifetime checking (not mutability tracking).</p>



<a name="208694634"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/208694634" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#208694634">(Sep 01 2020 at 14:28)</a>:</h4>
<p>whoops I missed there was an MCP - I think this should be warn-by-default until the false positive is fixed.</p>



<a name="210798176"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/210798176" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> triagebot <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#210798176">(Sep 21 2020 at 20:51)</a>:</h4>
<p><span class="user-group-mention" data-user-group-id="492">@T-compiler</span>: Proposal <a href="https://github.com/rust-lang/compiler-team/issues/346#issuecomment-696368765">#346</a> has been seconded, and will be approved in 10 days if no objections are raised.</p>



<a name="211899261"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/233931-t-compiler/major%20changes/topic/Uplift%20%60temporary-cstring-as-ptr%60%20lint%20fr%E2%80%A6%20compiler-team%23346/near/211899261" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> triagebot <a href="https://rust-lang.github.io/zulip_archive/stream/233931-t-compiler/major-changes/topic/Uplift.20.60temporary-cstring-as-ptr.60.20lint.20fr.E2.80.A6.20compiler-team.23346.html#211899261">(Oct 01 2020 at 12:04)</a>:</h4>
<p>This proposal has been accepted: <a href="https://github.com/rust-lang/compiler-team/issues/346">#346</a>.</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>